Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nur Einträge mit Status 1 oder 0 ausgeben #64

Merged
merged 3 commits into from
Mar 3, 2024
Merged

Nur Einträge mit Status 1 oder 0 ausgeben #64

merged 3 commits into from
Mar 3, 2024

Conversation

eaCe
Copy link
Member

@eaCe eaCe commented Mar 3, 2024

Es werden nur noch Einträge die veröffentlicht sind ausgegeben, oder Einträge die geplant sind und deren Datum und Uhrzeit erreicht ist.

eaCe and others added 2 commits March 3, 2024 13:57
Es werden nur noch Einträge die veröffentlicht sind ausgegeben, oder Einträge die geplant sind und deren Datum und Uhrzeit erreicht ist.
@alxndr-w
Copy link
Member

alxndr-w commented Mar 3, 2024

Wieso Status 0?

Du kannst übrigens auch where('status', 1, '>=') statt whereRaw(...) verwenden

@eaCe
Copy link
Member Author

eaCe commented Mar 3, 2024

Natürlich hast du recht. 0 weil geplant.

@eaCe
Copy link
Member Author

eaCe commented Mar 3, 2024

@alxndr-w halt. Sorry 0 ist geplant, 1 ist veröffentlicht. Aber -1 wäre nun auch dabei. Das ist doch auch falsch

@alxndr-w
Copy link
Member

alxndr-w commented Mar 3, 2024

Wieso wäre -1 da dabei?

Geplante Beiträge sollten nicht dargestellt werden. Höchstens vielleicht Mal, wenn man eingeloggt wäre. Alles ab 1 aufwärts ist richtig und so ist es doch auch in deinem Commit, oder nicht?

@eaCe
Copy link
Member Author

eaCe commented Mar 3, 2024

Sollte geplant nicht dabei sein, wenn das Datum eingetreten ist?

@alxndr-w
Copy link
Member

alxndr-w commented Mar 3, 2024

Doch, aber durch den Cronjob wird der Status ja dann auf 1 geändert und damit eine URL erzeugt.

@alxndr-w alxndr-w merged commit 0d84312 into main Mar 3, 2024
1 check passed
@alxndr-w alxndr-w deleted the eaCe-patch-1 branch March 3, 2024 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants